Skip to content

Conversation

@jstac
Copy link
Contributor

@jstac jstac commented Apr 21, 2023

No description provided.

@netlify
Copy link

netlify bot commented Apr 21, 2023

Deploy Preview for taupe-gaufre-c4e660 ready!

Name Link
🔨 Latest commit c0806f5
🔍 Latest deploy log https://app.netlify.com/sites/taupe-gaufre-c4e660/deploys/64444ef3a3b21c000890fd18
😎 Deploy Preview https://deploy-preview-168--taupe-gaufre-c4e660.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@github-actions
Copy link

github-actions bot commented Apr 21, 2023

@jstac
Copy link
Contributor Author

jstac commented Apr 21, 2023

Would someone mind to look into this build error for me?

@jstac jstac added the ready label Apr 22, 2023
@jstac
Copy link
Contributor Author

jstac commented Apr 22, 2023

@Smit-create @shlff With apologies, I ended up cutting the quasi-linear exercise. Thanks for your efforts trying to make it work. Nonetheless, it was hard to clean up and it's a kind of unusual case. So I think what we have now is good.

Perhaps @shlff could briefly review this and then @mmcky could merge? There's also a build error --- could someone please look into that (@mmcky or @Smit-create , if you have time?)

@Smit-create
Copy link
Member

Thanks, @jstac for the fixes. I'm having a look at the failures.

@Smit-create
Copy link
Member

This is ready to go in after approvals from @shlff and @mmcky. Thank you, John!

Copy link
Member

@shlff shlff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @jstac and @Smit-create . This lecture looks very beautiful now.

Also great thanks for John's extremely helpful advice on academic writing.

I quickly reviewed the lecture. Please find my comments above and feel free to let me know if you like me to make any of those proposed changes @jstac .

```

Let's observe the dynamics of the equilibrium price $R^*_{t+1}$.
Let's redo our plot above but now inserting the equilibrium quantity and price.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

inserting -> insert?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Inserting is correct.

lectures/olg.md Outdated
cannot be obtained analytically.
To solve the equation we need to turn to Newton's method.
Instead, solve for $k^*$ using newton's method.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

newton's -> Newton's

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. Also, "Instead, we solve".

lectures/olg.md Outdated
:label: crra_newton_2
g(k^*) = k^* \left [ 1 + \beta^{-1/\gamma} (\alpha (k^*)^{\alpha-1})^{(\gamma-1)/\gamma} \right ] - (1-\alpha)(k^*)^{\alpha}
```
First let define $f$.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let define -> let's define

lectures/olg.md Outdated

### Consumer's problem

Suppose that utility for individuals born at time $t$ take the form
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

take the form -> takes the form

lectures/olg.md Outdated
The first-order conditions for a maximum can be obtained by taking the derivative of the objective function with respect to capital and labor respectively and setting it to zero:
### Demand

Using our assumption $\ell_1 = 1$ allows us to write
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

$\ell_1 = 1$ -> ``$\ell_t = 1$

ax.plot(k_grid, k_grid_next, lw=2, alpha=0.6, label='$g$')
ax.plot(k_grid, k_grid, 'k-', lw=1, alpha=0.7, label='$45^{\circ}$')
ax.plot(k_grid, k_grid_next, lw=2, alpha=0.6, label='$g$')
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to define $g$ in the math expression above.

ax.plot(np.arange(ts_length), np.full(ts_length, k_star),
alpha=0.6, color='red', label=r'$k^*$')
ax.legend(fontsize=10)
ax.plot(k_grid, k_grid_next, lw=2, alpha=0.6, label='$g$')
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here.

lectures/olg.md Outdated
```{code-cell} ipython3
def find_Rstar(K_prev, model):
return optimize.newton(find_Rstar_newton, 0.5, args=(K_prev, model))
We introduce a function $g$ such that
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we consider using other notations than $g$ to distinguish it from the $g$ used in the plot code above?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good idea. I've changed it to $h$.

@jstac
Copy link
Contributor Author

jstac commented Apr 22, 2023

Thanks @shlff for very helpful comments. I've made those changes in c0806f5.

@shlff
Copy link
Member

shlff commented Apr 23, 2023

Thanks @shlff for very helpful comments. I've made those changes in c0806f5.

Thanks for your nice work @jstac . I've reviewed this lecture again and it looks very good to me now.

I will leave it to you or @mmcky for merging.

@jstac
Copy link
Contributor Author

jstac commented Apr 23, 2023

Thanks @shlff , much appreciated. Merging.

@jstac jstac merged commit 2045e81 into main Apr 23, 2023
@jstac jstac deleted the update-olg branch April 23, 2023 21:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants